Skip to content

test: MistalChatGenerator - relax test_live_run_with_tools test#2965

Merged
anakin87 merged 1 commit intomainfrom
relax-mistral-tools-test
Mar 17, 2026
Merged

test: MistalChatGenerator - relax test_live_run_with_tools test#2965
anakin87 merged 1 commit intomainfrom
relax-mistral-tools-test

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Mar 17, 2026

Related Issues

Now it seems that a text is returned, in addition to the tool call

Proposed Changes:

  • relax the test: remove assert message.text == ""

How did you test it?

CI

Checklist

@anakin87 anakin87 marked this pull request as ready for review March 17, 2026 07:58
@anakin87 anakin87 requested a review from a team as a code owner March 17, 2026 07:58
@anakin87 anakin87 requested review from bogdankostic and removed request for a team March 17, 2026 07:58
Copy link
Copy Markdown
Contributor

@sjrl sjrl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@anakin87 anakin87 merged commit 3d1c245 into main Mar 17, 2026
12 checks passed
@anakin87 anakin87 deleted the relax-mistral-tools-test branch March 17, 2026 08:16
@anakin87 anakin87 self-assigned this Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants